-
Notifications
You must be signed in to change notification settings - Fork 296
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Strava: BackcountrySki mapping #314
base: master
Are you sure you want to change the base?
Conversation
Mapping BackcountrySki to CrossCountrySkiing, instead of DownhillSkiing.
Hi, short story, when syncing Ski touring activities from Strava to Endomondo they end up as Downhill skiing, which is completely wrong ;-) This patch fixes that, with the currently available activities in ActivityType type. However, I wish there was a better distinction between snow skiing sports. The nomenclature is not standardized and differs in different services. I would divide this depending on the equipment used (and this defines terrain type as well):
So I would create at least this: a new ActivityType.BackcountrySki and map it to: "Backcountry Ski" in Strava, "Ski touring" in Endomondo. Not sure about the other services though. What do you think? Thanks, |
I get it, managing all the mappings must be a PITA for you...
Even so, I think mapping "BackcountrySki" to DownhillSkiing does not make
sense (logically). Similar to mapping "NordicSki" to DownhillSkiing, which
was fixed in 203ed01.
2017-04-22 20:46 GMT+02:00 Collin Fair <[email protected]>:
… This gets changed around every few years -
203ed01
<203ed01>,
0b84182
<0b84182>
:|
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#314 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAKyT_b6nHlQH5Rusd9ckAFjG_sMUhpbks5ryksigaJpZM4M1Q1O>
.
|
I have to agree with trozen moreover they became "Resort Skiing" in Garmin connect and it has nothing to do with "BackcountrySki" or "Ski touring" whatever you intend it them to mean. Please merge this PR |
da1b4e7
to
1081891
Compare
Mapping BackcountrySki to CrossCountrySkiing, instead of DownhillSkiing.